Skip to content

[Liquidity Mining] Obligation realloc (8) #207

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 17 commits into from
Apr 6, 2025
Merged

[Liquidity Mining] Obligation realloc (8) #207

merged 17 commits into from
Apr 6, 2025

Conversation

@vanitymnm
Copy link

@vanitymnm vanitymnm commented Apr 1, 2025 •
edited
Loading

  • Reserve is now packed and unpacked less times per each ix. This improves our CU budget. State tracking now leverages runtime borrow checking to avoid double write or outdated reads bugs
  • Obligation account is reallocated to fit the necessary dynamically computed size
  • Improve realloc tests calculations
  • Fix bug in the shares logic
  • add tests for user shares attribution

@vanitymnm vanitymnm added the liquidity mining Relates to liquidity mining featgure label Apr 1, 2025
@vanitymnm vanitymnm self-assigned this Apr 1, 2025
@vanitymnm vanitymnm mentioned this pull request Apr 1, 2025
Open
vanitymnm
vanitymnm commented Apr 5, 2025
View reviewed changes
@vanitymnm vanitymnm marked this pull request as ready for review April 5, 2025 19:10
@codecov
Copy link

codecov bot commented Apr 6, 2025

Codecov Report

Attention: Patch coverage is 92.81998% with 69 lines in your changes missing coverage. Please review.

Please upload report for BASE (feat/liquidity-mining@590c58c). Learn more about missing BASE report.

Files with missing lines Patch % Lines
...rc/processor/liquidity_mining/claim_user_reward.rs 0.00% 30 Missing ⚠️
...en-lending/program/src/processor/account_borrow.rs 84.31% 16 Missing ⚠️
token-lending/sdk/src/state/liquidity_mining.rs 95.26% 10 Missing ⚠️
...-lending/program/src/processor/liquidity_mining.rs 0.00% 7 Missing ⚠️
token-lending/program/src/processor.rs 98.54% 3 Missing ⚠️
...c/processor/liquidity_mining/cancel_pool_reward.rs 0.00% 2 Missing ⚠️
.../src/processor/liquidity_mining/add_pool_reward.rs 0.00% 1 Missing ⚠️
Additional details and impacted files 
@@                   Coverage Diff                    @@
##             feat/liquidity-mining     #207   +/-   ##
========================================================
  Coverage                         ?   83.12%           
========================================================
  Files                            ?       66           
  Lines                            ?    25551           
  Branches                         ?        0           
========================================================
  Hits                             ?    21240           
  Misses                           ?     4311           
  Partials                         ?        0

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@vanitymnm vanitymnm merged commit 4e58ca7 into feat/liquidity-mining Apr 6, 2025
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

@vanitymnm vanitymnm

Labels

liquidity mining Relates to liquidity mining featgure

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@vanitymnm